Skip to content

Task/#104 net 10 upgrade#106

Merged
StuartFerguson merged 2 commits intomasterfrom
task/#104_net_10_upgrade
Dec 8, 2025
Merged

Task/#104 net 10 upgrade#106
StuartFerguson merged 2 commits intomasterfrom
task/#104_net_10_upgrade

Conversation

@StuartFerguson
Copy link
Copy Markdown
Member

closes #104
closes #105
closes #103

Comment thread .github/workflows/createrelease.yml Outdated
Comment on lines 91 to 121

Check warning

Code scanning / CodeQL

Workflow does not contain permissions Medium

Actions job or workflow does not limit the permissions of the GITHUB_TOKEN. Consider setting an explicit permissions block, using the following as a minimal starting point: {}

Copilot Autofix

AI 5 months ago

To fix this issue, explicitly add a permissions block at the root of the workflow file. Begin by setting the permissions to the lowest possible value (read-only access for contents), and elevate permissions only in those jobs or steps that genuinely require them (such as jobs using actions that upload or download artifacts, though most such actions only require contents: read). Since the workflow mainly restores/builds code, interacts with docker, and uploads/downloads artifacts, contents: read will likely suffice. Set this block immediately after the name: declaration and before the on: block to ensure it applies globally to all jobs, unless overridden by a job-level permissions block.


Suggested changeset 1
.github/workflows/createrelease.yml

Autofix patch

Autofix patch
Run the following command in your local git repository to apply this patch
cat << 'EOF' | git apply
diff --git a/.github/workflows/createrelease.yml b/.github/workflows/createrelease.yml
--- a/.github/workflows/createrelease.yml
+++ b/.github/workflows/createrelease.yml
@@ -1,4 +1,6 @@
 name: Release
+permissions:
+  contents: read
 
 on:
   release:
EOF
@@ -1,4 +1,6 @@
name: Release
permissions:
contents: read

on:
release:
Copilot is powered by AI and may make mistakes. Always verify output.
@StuartFerguson StuartFerguson merged commit a708371 into master Dec 8, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move to Windows Release Workflow Net 10 Upgrade Update Shared Nugets

2 participants